Skip to content

fix guidance_scale value not equal to the value in comments#8941

Merged
yiyixuxu merged 2 commits intohuggingface:mainfrom
efwfe:patch-1
Jul 25, 2024
Merged

fix guidance_scale value not equal to the value in comments#8941
yiyixuxu merged 2 commits intohuggingface:mainfrom
efwfe:patch-1

Conversation

@efwfe
Copy link
Copy Markdown
Contributor

@efwfe efwfe commented Jul 23, 2024

fix guidance_scale value not equal to the value in comments

default guidance_scale value:
image

comment guidance_scale value:
image

What does this PR do?

Fixes # (issue)

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

num_inference_steps: int = 28,
timesteps: List[int] = None,
guidance_scale: float = 7.0,
guidance_scale: float = 5.0,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! can you change the comments (doc string) instead? we prefer not to change the default value because it will cause backward breaking

Copy link
Copy Markdown
Contributor Author

@efwfe efwfe Jul 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, already updated ~

@efwfe efwfe closed this Jul 24, 2024
@efwfe efwfe reopened this Jul 24, 2024
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@yiyixuxu yiyixuxu merged commit 9b8c860 into huggingface:main Jul 25, 2024
@efwfe efwfe deleted the patch-1 branch July 26, 2024 07:04
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
fix guidance_scale value not equal with the value in comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants